sysbuild: Align 53bsim with 53 SoC #24742

rugeGerritsen · 2025-09-29T09:10:50Z

This PR unifies those boards which allows us to cleanup some tests.
See commit messages for additional details.

NordicBuilder · 2025-09-29T09:18:59Z

CI Information

^{To view the history of this post, clich the 'edited' button above}
Build number: 8

Inputs:

Sources:

sdk-nrf: PR head: 43e798b3285a2ccf85d6b6e287f305dff0a498eb

more details

sdk-nrf:

PR head: 43e798b3285a2ccf85d6b6e287f305dff0a498eb
merge base: 5999ecf877bf2f926d7b83a03e79bff725a89d84
target head (main): 9710cb1e0290d18f7f0422c72c1caffd4b6725e4
Diff

Github labels

Enabled	Name	Description
	ci-disabled	Disable the ci execution
	ci-all-test	Run all of ci, no test spec filtering will be done
	ci-force-downstream	Force execution of downstream even if twister fails
	ci-run-twister	Force run twister
	ci-run-zephyr-twister	Force run zephyr twister

List of changed files detected by CI (6)

sysbuild
│  ├── Kconfig.netcore
│  │ netcore.cmake
tests
│  ├── bluetooth
│  │  ├── bsim
│  │  │  ├── nrf_auraconfig
│  │  │  │  ├── Kconfig.sysbuild
│  │  │  │  ├── sysbuild.cmake
│  │  │  │  ├── tester
│  │  │  │  │  ├── Kconfig.sysbuild
│  │  │  │  │  │ sysbuild.cmake

Outputs:

Toolchain

Version: a7529a11f4
Build docker image: docker-dtr.nordicsemi.no/sw-production/ncs-build:a7529a11f4_bba2ea5f2e

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

◻️ Toolchain - Skipped: existing toolchain is used
✅ Build twister - Skipped: Skipping Build & Test as it succeeded in a previous run: 7
✅ Integration tests
- ✅ test-sdk-audio - Skipped: Job was skipped as it succeeded in a previous run
- ✅ desktop52_verification - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-apps - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-ble_mesh - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-ble_samples - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-chip - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-nfc - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-nrf-iot_cloud - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-nrf-iot_libmodem-nrf - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-nrf-iot_serial_lte_modem - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-nrf-iot_zephyr_lwm2m - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-nrf-iot_samples - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-nrf-iot_lwm2m - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-nrf-iot_thingy91 - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-nrf_crypto - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-proprietary_esb - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-rpc - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-rs - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-fem - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-tfm - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-thread-main - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-sdk-find-my - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-nrf_lrcs_mosh - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-nrf_lrcs_positioning - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-sdk-wifi - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-low-level
- ✅ test-sdk-pmic-samples - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-sdk-mcuboot - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-sdk-dfu - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-fw-nrfconnect-ps-main - Skipped: Job was skipped as it succeeded in a previous run
- ✅ test-secdom-samples-public - Skipped: Job was skipped as it succeeded in a previous run

Note: This message is automatically posted and updated by the CI

NordicBuilder · 2025-09-29T10:48:30Z

Memory footprint analysis revealed the following potential issues

applications.nrf_desktop.zdebug.uart[[email protected]/nrf54h20/cpuapp]: RAM size increased by 23661[B] in comparison to the main[5999ecf] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zdebug.uart[[email protected]/nrf54h20/cpuapp]: ROM size increased by 13752[B] in comparison to the main[5999ecf] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[[email protected]/nrf54h20/cpuapp]: RAM size increased by 23657[B] in comparison to the main[5999ecf] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.nrf_desktop.zrelease[[email protected]/nrf54h20/cpuapp]: ROM size increased by 12044[B] in comparison to the main[5999ecf] branch. - link (cc: @nrfconnect/ncs-si-bluebagel)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 9102[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 5858[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High RAM usage: 12744[B] - link (cc: @nrfconnect/ncs-ll-ursus)
applications.hpf.gpio.icbmsg[nrf54l15dk/nrf54l15/cpuflpr]: High ROM usage: 9492[B] - link (cc: @nrfconnect/ncs-ll-ursus)

Note: This message is automatically posted and updated by the CI (latest/sdk-nrf/PR-24742/7)

nordicjm · 2025-09-30T10:27:17Z

sysbuild/Kconfig.netcore

 config SUPPORT_NETCORE
 	bool
-	default y if (SOC_NRF5340_CPUAPP || SOC_NRF54H20_CPUAPP)
+	default y if (SOC_NRF5340_CPUAPP || SOC_NRF54H20_CPUAPP || BOARD_NRF5340BSIM)


@aescolar should this be added here?

From the POV of what the PR is doing I do not see any issue right away.
From the point of view of the actual kconfig option used, as this is a sysbuild file, I guess we are limited to not using the COMPATIBLE options, but that maybe BOARD_NRF5340BSIM_NRF5340_CPUAPP is a bit better.

aescolar · 2025-09-30T11:17:50Z

sysbuild/Kconfig.netcore

 config SUPPORT_NETCORE
 	bool
-	default y if (SOC_NRF5340_CPUAPP || SOC_NRF54H20_CPUAPP)
+	default y if (SOC_NRF5340_CPUAPP || SOC_NRF54H20_CPUAPP || BOARD_NRF5340BSIM)


From the POV of what the PR is doing I do not see any issue right away.
From the point of view of the actual kconfig option used, as this is a sysbuild file, I guess we are limited to not using the COMPATIBLE options, but that maybe BOARD_NRF5340BSIM_NRF5340_CPUAPP is a bit better.

tests/bluetooth/bsim/nrf_auraconfig/tester/sysbuild.cmake

aescolar

lgtm

nordicjm · 2025-10-01T07:53:03Z

sysbuild/netcore.cmake


  set_property(GLOBAL PROPERTY PM_DOMAINS ${PM_DOMAINS})
+
+  if (SB_CONFIG_BOARD_NRF5340BSIM_NRF5340_CPUAPP)


Suggested change

if (SB_CONFIG_BOARD_NRF5340BSIM_NRF5340_CPUAPP)

if(SB_CONFIG_BOARD_NRF5340BSIM_NRF5340_CPUAPP)

Created zephyrproject-rtos/zephyr#96846 to address the underlying issue: Style checks not catching this

This commit sets defaults for the sysbuild configurations SUPPORT_NETCORE and NETCORE_REMOTE_BOARD_TARGET_CPUCLUSTER so that it behaves more similar to the actual 5340 soc. This allows us to remove explicit adding of network core images. Signed-off-by: Rubin Gerritsen <[email protected]>

Use NRF_DEFAULT_IPC_RADIO to enable building of the hci_ipc radio sample for the network core instead of adding it manually. Signed-off-by: Rubin Gerritsen <[email protected]>

rugeGerritsen requested a review from a team as a code owner September 29, 2025 09:10

NordicBuilder added the changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. label Sep 29, 2025

rugeGerritsen force-pushed the 53bsim_netcore_sysbuild branch from cab03ee to 464ad69 Compare September 29, 2025 09:13

rugeGerritsen force-pushed the 53bsim_netcore_sysbuild branch from 464ad69 to 04ff26c Compare September 30, 2025 07:23

rugeGerritsen requested a review from a team as a code owner September 30, 2025 07:23

rugeGerritsen requested review from koffes, rick1082, alexsven and gWacey September 30, 2025 07:23

rugeGerritsen force-pushed the 53bsim_netcore_sysbuild branch from 04ff26c to 5b23fff Compare September 30, 2025 07:24

nordicjm reviewed Sep 30, 2025

View reviewed changes

aescolar reviewed Sep 30, 2025

View reviewed changes

rugeGerritsen force-pushed the 53bsim_netcore_sysbuild branch 2 times, most recently from 2e60584 to 8c5bc5a Compare October 1, 2025 07:46

rugeGerritsen requested review from aescolar and nordicjm October 1, 2025 07:46

aescolar reviewed Oct 1, 2025

View reviewed changes

nordicjm requested changes Oct 1, 2025

View reviewed changes

rugeGerritsen added 2 commits October 1, 2025 09:58

tests: nrf_auracast: Simplify network core building

43e798b

Use NRF_DEFAULT_IPC_RADIO to enable building of the hci_ipc radio sample for the network core instead of adding it manually. Signed-off-by: Rubin Gerritsen <[email protected]>

rugeGerritsen force-pushed the 53bsim_netcore_sysbuild branch from 8c5bc5a to 43e798b Compare October 1, 2025 07:58

rugeGerritsen requested a review from nordicjm October 1, 2025 07:58

nordicjm approved these changes Oct 1, 2025

View reviewed changes

alexsven approved these changes Oct 1, 2025

View reviewed changes

rugeGerritsen merged commit ed14049 into nrfconnect:main Oct 1, 2025
13 checks passed

rugeGerritsen deleted the 53bsim_netcore_sysbuild branch October 1, 2025 12:20

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

sysbuild: Align 53bsim with 53 SoC #24742

sysbuild: Align 53bsim with 53 SoC #24742

Uh oh!

rugeGerritsen commented Sep 29, 2025

Uh oh!

NordicBuilder commented Sep 29, 2025 •

edited

Loading

Uh oh!

NordicBuilder commented Sep 29, 2025 •

edited

Loading

Uh oh!

nordicjm Sep 30, 2025

Uh oh!

aescolar Sep 30, 2025

Uh oh!

aescolar Sep 30, 2025

Uh oh!

Uh oh!

aescolar left a comment

Uh oh!

nordicjm Oct 1, 2025

Uh oh!

rugeGerritsen Oct 1, 2025

Uh oh!

Uh oh!

Uh oh!


		set_property(GLOBAL PROPERTY PM_DOMAINS ${PM_DOMAINS})

		if (SB_CONFIG_BOARD_NRF5340BSIM_NRF5340_CPUAPP)

	if (SB_CONFIG_BOARD_NRF5340BSIM_NRF5340_CPUAPP)
	if(SB_CONFIG_BOARD_NRF5340BSIM_NRF5340_CPUAPP)

sysbuild: Align 53bsim with 53 SoC #24742

sysbuild: Align 53bsim with 53 SoC #24742

Uh oh!

Conversation

rugeGerritsen commented Sep 29, 2025

Uh oh!

NordicBuilder commented Sep 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

CI Information

Inputs:

Sources:

Github labels

Outputs:

Toolchain

Test Spec & Results: ✅ Success; ❌ Failure; 🟠 Queued; 🟡 Progress; ◻️ Skipped; ⚠️ Quarantine

Uh oh!

NordicBuilder commented Sep 29, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Memory footprint analysis revealed the following potential issues

Uh oh!

nordicjm Sep 30, 2025

Choose a reason for hiding this comment

Uh oh!

aescolar Sep 30, 2025

Choose a reason for hiding this comment

Uh oh!

aescolar Sep 30, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

aescolar left a comment

Choose a reason for hiding this comment

Uh oh!

nordicjm Oct 1, 2025

Choose a reason for hiding this comment

Uh oh!

rugeGerritsen Oct 1, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

NordicBuilder commented Sep 29, 2025 •

edited

Loading

NordicBuilder commented Sep 29, 2025 •

edited

Loading